Skip to content

feat: modules support - #296

Open
alienvspredator wants to merge 5 commits into
mainfrom
feature/extension-support
Open

feat: modules support#296
alienvspredator wants to merge 5 commits into
mainfrom
feature/extension-support

Conversation

@alienvspredator

@alienvspredator alienvspredator commented May 18, 2026 •
edited
Loading

Copy link
Copy Markdown
Contributor

What this PR does / why we need it:

Add modules support to the session manager, define and implement standard `trust` and `database` modules.

Special notes for your reviewer:

The modules are statically linked at built-time.
Custom-built session manager with other modules can be built by altering imports in the main.go file. The main() body remains unchanged (makes a call to maincmd.Main()).

Release note:

- Support statically linked modules at build time.
- Implement `trust` module with OIDC.
- Implement `database` module with pgxpool.

feat: module extensions

@coderabbitai

coderabbitai Bot commented May 18, 2026 •
edited
Loading

Copy link
Copy Markdown

Important

Review skipped

Auto reviews are disabled on this repository. Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro Plus

Run ID: d787f1a6-df76-4854-902c-61ccfd026992

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

  • 🔍 Trigger review

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@alienvspredator
alienvspredator force-pushed the feature/extension-support branch from 85fbad9 to f0e17e2 Compare May 18, 2026 09:57
@alienvspredator
alienvspredator force-pushed the feature/extension-support branch 8 times, most recently from 710cacf to ba47cb2 Compare May 18, 2026 13:30
@openkcm openkcm deleted a comment from coderabbitai Bot May 18, 2026
@alienvspredator
alienvspredator force-pushed the feature/extension-support branch 7 times, most recently from 70549ee to 47d3a66 Compare May 27, 2026 08:50
cb80
cb80 requested changes May 27, 2026
View reviewed changes
Comment thread modules/oidctrust/module.go Outdated
Comment thread internal/config/config.go
"time"

"github.com/creasty/defaults"
"github.com/knadh/koanf/v2"

@cb80 cb80 May 27, 2026

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We usually use viper for this, so do we really need koanf?
Especially as one of its deps github.com/mitchellh/copystructure is archived.

@alienvspredator alienvspredator Jun 2, 2026

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We can't use viper for this use-case. Viper can't parse and store ambiguous values, as well as some issues with Viper:

So we can't really use Viper here.

Comment thread context.go Outdated
@alienvspredator
alienvspredator force-pushed the feature/extension-support branch from 47d3a66 to 8f02e4c Compare June 2, 2026 07:43
@alienvspredator
alienvspredator force-pushed the feature/extension-support branch 2 times, most recently from 0758404 to 299e11b Compare June 23, 2026 07:20
@alienvspredator
alienvspredator requested a review from cb80 June 23, 2026 08:21
@alienvspredator
alienvspredator force-pushed the feature/extension-support branch from 299e11b to dd47031 Compare June 23, 2026 11:50
@alienvspredator
alienvspredator force-pushed the feature/extension-support branch from dd47031 to 41d81f6 Compare July 28, 2026 19:13
@alienvspredator
alienvspredator force-pushed the feature/extension-support branch from 41d81f6 to c41eb72 Compare July 29, 2026 06:45
@alienvspredator
feat: module extensions
0f71db6 
Signed-off-by: Danylo Shevchenko <danilshevchenko123456@gmail.com>
@alienvspredator
feat: application lifecycle
e261728 
Signed-off-by: Danylo Shevchenko <danilshevchenko123456@gmail.com>
@alienvspredator
refactor: rename mapping into trust
a8fe439 
Signed-off-by: Danylo Shevchenko <danilshevchenko123456@gmail.com>
@alienvspredator
feat: grpc app module
5192e2e 
Signed-off-by: Danylo Shevchenko <danilshevchenko123456@gmail.com>
@alienvspredator
alienvspredator force-pushed the feature/extension-support branch from c41eb72 to f2e796a Compare July 29, 2026 06:56
cb80
cb80 requested changes Jul 29, 2026
View reviewed changes

@cb80 cb80 left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please bump the chart version.

@alienvspredator
feat: oidcmapping module
dcb2875 
Signed-off-by: Danylo Shevchenko <danilshevchenko123456@gmail.com>
@alienvspredator
alienvspredator force-pushed the feature/extension-support branch from f2e796a to dcb2875 Compare July 29, 2026 07:13
@alienvspredator
alienvspredator requested a review from cb80 July 29, 2026 07:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@cb80 cb80 Awaiting requested review from cb80

Requested changes must be addressed to merge this pull request.

Assignees

No one assigned

Labels

dependencies dev-ops feature helm-chart tests

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@alienvspredator @cb80